-
Notifications
You must be signed in to change notification settings - Fork 7.8k
boards: fobe: add FoBE Quill nRF52840 Mesh board #94292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
boards: fobe: add FoBE Quill nRF52840 Mesh board #94292
Conversation
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
b58689e
to
7f99b67
Compare
7f99b67
to
10d3432
Compare
boards/fobe/quill_nrf52840_mesh/doc/img/quill_nrf52840_mesh_pinout.png
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/doc/img/quill_nrf52840_mesh.png
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh-pinctrl.dtsi
Outdated
Show resolved
Hide resolved
00a8b55
to
9cf22ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, could you please take a look at a couple nitpicks?
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
boards/fobe/quill_nrf52840_mesh/quill_nrf52840_mesh_common.dtsi
Outdated
Show resolved
Hide resolved
Why not? But I’m having some network issues, so I might be a bit late. |
9cf22ba
to
293761c
Compare
293761c
to
3f823bd
Compare
Fixed some workflow build warning. |
boards/fobe/quill_nrf52840_mesh/doc/img/quill_nrf52840_mesh.webp
Outdated
Show resolved
Hide resolved
@chihosin by the way, could you please fix your sign-off for compliance to pass? Do it like |
fdd4a7b
to
4fdc525
Compare
@chihosin Please refrain from marking request for changes as resolved unless it's for trivial suggestoin where you apply suggestions verbatim (which to be clear, is the case for several of them, but for the others it's just difficult for reviewers to come back and remember what they need to "re"-check) - https://docs.zephyrproject.org/latest/contribute/contributor_expectations.html#workflow-suggestions-that-help-reviewers |
boards/fobe/quill_nrf52840_mesh/doc/img/quill_nrf52840_mesh.webp
Outdated
Show resolved
Hide resolved
This commit introduces the FoBE Quill nRF52840 Mesh board configuration, including device tree files, Kconfig settings, and necessary documentation. It also adds GPIO header definitions and updates vendor prefixes for proper identification. Signed-off-by: Chiho Sin <[email protected]>
4fdc525
to
3647e5a
Compare
|
This commit introduces the FoBE Quill nRF52840 Mesh board configuration, including device tree files, Kconfig settings, and necessary documentation. It also adds GPIO header definitions and updates vendor prefixes for proper identification.
Tested on zephyr samples.